Add GitHub workflows and templates - #45
Merged
Merged
Conversation
- Added .github/workflows/pr_ci_report.py to create a markdown file from results of Pull Request CI Report workflow to be used as auto-generated comment in repo.
- Added .github/workflows/pr-ci-report.yml to define a workflow for running Python and R tests automatically for a pull request, commenting the results as a table.
- Added .github/workflows/pr-version-check.yml to check that any pull requests raised on merged are increasing the version number.
- Added .github/workflows/publish-release.yml to build source distributions and publish release to GitHub on pushes to main.
- Modified .github/workflows/pr-ci-report.yml to fix incorrect R dependency installation path, and to use opencv-python-headless during Python tests to ensure they run on Linux.
- Modified .github/workflows/pr-ci-report.yml to fix missing testthat installation and to install the dependencies required by Qt on Linux.
- Modified tests/r/test_utils_import.R to correct R import tests which were not using the correct filenames for spectral count results files, causing issues with column names being imported.
- Modified .github/workflows/pr-ci-report.yml to include checkout step for posting summary comment so the pr_ci_report.py script is accessible.
Pull Request CI ReportCaution GitHub Actions runners do not have ThermoRawFileParser or Crux installed. Tests dependent on these binaries are skipped and should instead be run manually before merging. Summary
Python TestsPython versions tested: 3.14, 3.15
R TestsR versions tested: 4.4.3, 4.5.3, 4.6.1
Auto-generated by the comMS Pull Request CI Report workflow. See full logs in the Actions run for this PR. |
- Modified .github/workflows/pr-ci-report.yml to download required Linux dependencies for R packages.
- Modified .github/workflows/pr-ci-report.yml to include additional Linux dependencies for running R scripts, and to add debug for report comment.
- Modified .github/workflows/pr-ci-report.yml to add debug step and remove R version 4.3.0 from matrix.
- Modified .github/workflows/pr-ci-report.yml to add additional Linux dependencies. - Modified .github/workflows/pr_ci_report.py to fix glob pattern. - Modified README.md to update minimum R version required.
- Modified .github/workflows/pr-ci-report.yml to update post comment job so checkout runs first, to stop deletion of downloaded artifacts as before.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull request (PR) description:
This PR adds the following GitHub resources:
mainbranch increase the version number.uv build) and publishing a tagged release with these upon pushed to themainbranch.Issues resolved:
n/a